Add performance test model and update pathfinder tests#42
Conversation
| - predicate_id | ||
| - predicate_name |
There was a problem hiding this comment.
If predicates are included, qualifiers have to be as well.
| @@ -243,7 +244,12 @@ classes: | |||
| be defined as filters to specify TestAsset membership in a "Block List" collection. | |||
|
|
|||
| PathfinderTestCase: | |||
There was a problem hiding this comment.
Should this not model closely to TestCase? If TestCase includes input category and id, then I think this should as well, split into source and target as it is in the test asset.
There was a problem hiding this comment.
I can add it back in, but from the Test Harness perspective, these are unused fields and redundant in the Test Assets. I'm inclined to leave them out. What are your thoughts?
There was a problem hiding this comment.
I think from a test harness perspective, leaving them out is totally fine. I was really only suggesting it for consistency's sake, but the test runners don't make use of those fields at all.
There was a problem hiding this comment.
Let's leave them out for now
2 participants
No description provided.